Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feat: BLS signatures over BLS12-381 curve #176

Open
wants to merge 29 commits into
base: community-edition
Choose a base branch
from
Open

feat: BLS signatures over BLS12-381 curve #176

wants to merge 29 commits into from
+1,559 −23

Conversation

nulltea
Copy link

@nulltea nulltea commented Oct 2, 2023 •
edited
Loading

Add BLS signatures for newly added BLS12-381. Relies on axiom-crypto/halo2curves#8. It should be merged after #175.

Open questions:

  • is there a need for BlsSignatureChip trait to standardize for BN256 and BLS12-381 curves?

Benchmarks

test_bls_signature (verify 1 BLS signature w/o hash to curve) k=19 lookup_bits=18:

Gate Chip | Phase 0: 4960683 advice cells
Total 82 fixed cells
Total range check advice cells to lookup per phase: [619032, 0, 0]

@nulltea nulltea mentioned this pull request Oct 9, 2023
Open
@nulltea
use separte halo2curves with halo2-pse feature
f689ab8
@nulltea
use separte halo2curves with halo2-pse feature
1a01328
@nulltea
Merge remote-tracking branch 'upstream/community-edition' into feat/b…
d7bbe5b 
…ls12-381-pairing
@nulltea
optimize final exp
d1d1b92
@nulltea
Merge branch 'feat/bls12-381-pairing' into feat/bls12-381-signatures
9452a6c
@nulltea
update test config
90b7594
@nulltea
update test config
b371954
@nulltea
fix bigint for bls12-381
ae455d8
@nulltea
fix bigint for bls12-381
9ce6725
@nulltea
Merge remote-tracking branch 'origin/community-edition' into feat/bls…
01654cf 
…12-381-pairing
@nulltea
fix build
8b79482
@nulltea
Merge branch 'feat/bls12-381-pairing' into feat/bls12-381-signatures
5775309
@nulltea
fix for halo2-pse
dd8b0d2
@nulltea
Merge branch 'feat/bls12-381-pairing' into feat/bls12-381-signatures
39994bf
@nulltea
fix ec_add test config
7ef05a1
@nulltea
Fix bls_signature_verify insecure usage (#2)
a01e898
@nulltea
Add must_use flag for is_valid_signature returned selector (#11)
938d6b7
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@nulltea